Skip to content

Conversation

compiler-errors
Copy link
Member

r? @Nilstrieb

We could also perhaps derive Ord on Edition and use comparison operators.

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Jul 16, 2023
@clarfonthey
Copy link
Contributor

We could also perhaps derive Ord on Edition and use comparison operators.

This seems way more reasonable to me.

@workingjubilee
Copy link
Member

Nils is /afk
r? compiler

@rustbot rustbot assigned oli-obk and unassigned Noratrieb Jul 19, 2023
Copy link
Contributor

@oli-obk oli-obk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

r=me with doc copy pasta fixed

}

/// Are we allowed to use features from the Rust 2018 edition?
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
/// Are we allowed to use features from the Rust 2018 edition?
/// Are we allowed to use features from the Rust 2021 edition?

}

/// Are we allowed to use features from the Rust 2018 edition?
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
/// Are we allowed to use features from the Rust 2018 edition?
/// Are we allowed to use features from the Rust 2024 edition?

@oli-obk oli-obk added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jul 19, 2023
@compiler-errors
Copy link
Member Author

@bors r=oli-obk rollup (naming only)

@bors
Copy link
Collaborator

bors commented Jul 19, 2023

📌 Commit 846cc63 has been approved by oli-obk

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Jul 19, 2023
bors added a commit to rust-lang-ci/rust that referenced this pull request Jul 19, 2023
Rollup of 7 pull requests

Successful merges:

 - rust-lang#113444 (add tests for alias bound preference)
 - rust-lang#113716 (Add the `no-builtins` attribute to functions when `no_builtins` is applied at the crate level.)
 - rust-lang#113754 (Simplify native_libs query)
 - rust-lang#113765 (Make it clearer that edition functions are `>=`, not `==`)
 - rust-lang#113774 (Improve error message when closing bracket interpreted as formatting fill character)
 - rust-lang#113785 (Fix invalid display of inlined re-export when both local and foreign items are inlined)
 - rust-lang#113803 (Fix inline_const with interpolated block)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit a47b7b0 into rust-lang:master Jul 19, 2023
@rustbot rustbot added this to the 1.73.0 milestone Jul 19, 2023
@compiler-errors compiler-errors deleted the at-least branch August 11, 2023 19:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants